-
Notifications
You must be signed in to change notification settings - Fork 11.9k
fix(@angular-devkit/build-angular): expand jest
and jest-environment-jsdom
to allow version 30
#30761
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
jest
and jest-environment-jsdom
to allow version 30jest
and jest-environment-jsdom
to allow version 30
…nt-jsdom` to allow version 30 This commit expands the peer deps of these dependencies to allow version 30. Closes angular#30760
|
||
export default async function (): Promise<void> { | ||
await applyJestBuilder(); | ||
await silentNpm('install', 'jest@29.5.0', 'jest-environment-jsdom@29.5.0', '--save-dev'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: Do we need to continue to support v29? Since this is still experimental, I don't think we're on the hook for backwards compatibility yet and it's probably reasonable to limit this to the latest major unless we have reason to believe that will be exceptionally difficult for users to adopt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So it seems that this would need further investigation to get version 30 to work. Let's have a chat about this offline to see the next steps for this builder.
@dgp1130 @alan-agius4 When support for Jest v30 could be merged? |
This would require some further investigation and work to land as some tests are failing with the latest jest version. |
@alan-agius4 Sure. I understand that. But the question is - when? It would be nice to know what are the plans for that support :) Don't want to check this PR every day :D |
on the right there is a "Subscribe" button where you can opt in to getting updates on the status and comments of this PR to avoid having to check manually. with that being said a timeline would be pretty cool as this is holding up my teams update to Jest 30 =) |
We haven't had a detailed investigation or discussion just yet. When we have more information to share, we'll share it. Also just a friendly reminder that the |
This commit expands the peer deps of these dependencies to allow version 30.
Closes #30760